Skip to content

Linter: Narrow html-no-self-closing offense location#1777

Merged
marcoroth merged 2 commits into
mainfrom
html-no-self-closing-location
Jun 5, 2026
Merged

Linter: Narrow html-no-self-closing offense location#1777
marcoroth merged 2 commits into
mainfrom
html-no-self-closing-location

Conversation

@marcoroth

@marcoroth marcoroth commented Jun 5, 2026

Copy link
Copy Markdown
Owner

This pull request updates the html-no-self-closing to update the offense location of the /> to only the node.tag_closing.location instead of the full node.location.

Before

CleanShot 2026-06-05 at 22 24 55@2x

After

CleanShot 2026-06-05 at 22 24 25@2x

@marcoroth marcoroth changed the title Linter: Narrow html-no-self-closing location Linter: Narrow html-no-self-closing offense location Jun 5, 2026
@pkg-pr-new

pkg-pr-new Bot commented Jun 5, 2026

Copy link
Copy Markdown
npx https://pkg.pr.new/@herb-tools/formatter@1777
npx https://pkg.pr.new/@herb-tools/language-server@1777
npx https://pkg.pr.new/@herb-tools/linter@1777

commit: eace39f

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

🌿 Interactive Playground and Documentation Preview

A preview deployment has been built for this pull request. Try out the changes live in the interactive playground:


🌱 Grown from commit eace39f


✅ Preview deployment has been cleaned up.

@marcoroth marcoroth merged commit 45dc69a into main Jun 5, 2026
20 of 21 checks passed
@marcoroth marcoroth deleted the html-no-self-closing-location branch June 5, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant